Skip to content
This repository was archived by the owner on Aug 1, 2024. It is now read-only.

chore: upgrade devstack edx-platform es to 7.10#813

Merged
connorhaugh merged 2 commits intomasterfrom
chore--upgrade-devstack-edx-platform-es-to-7.10
Aug 5, 2021
Merged

chore: upgrade devstack edx-platform es to 7.10#813
connorhaugh merged 2 commits intomasterfrom
chore--upgrade-devstack-edx-platform-es-to-7.10

Conversation

@connorhaugh
Copy link
Copy Markdown
Contributor

@connorhaugh connorhaugh commented Aug 5, 2021

chore: upgrade devstack edx-platform es to 7.10

As part of larger elasticsearch upgrade work, we must move edx-platform to use the 7.10 container.

This will ensure future compatibility and maybe make performance improvements to es calls.

Presently, edx-platofrm uses elasticsearch for searching within the content-library. Searching through the changelogs, no functionality should change in the present formation, as edx-platform es usage is relatively rudimentary.

Testing involved running unit, e2e, and hand testing of the creation and indexing of course content. Attention was paid to its usage in the teams api as well, as that is a notable edge usage.

Part of [TNL-8484](ES 7.10 - Upgrade and Test edx-platform Devstack on Elasticsearch 7.10) and the larger ES upgrade efforts:

https://openedx.atlassian.net/wiki/spaces/AC/pages/2923561087/Elasticsearch+7.10

Config changes: https://github.com/edx/configuration/pull/6496

@connorhaugh
Copy link
Copy Markdown
Contributor Author

Testing:
-make dev.pull to pull in config changes
-On Devstack, log in as edx@example.com
-In Studio create a some course content for the demo course containing the phrase "Purple Rain" in the unit title
-Publish said content
-Go to course home and search for course content containing "Purple Rain"
-View course content in search

Copy link
Copy Markdown
Contributor

@kdmccormick kdmccormick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So this will effectively drop any data that edx-platform has saved in elasticsearch, right? (not literally drop it, but the 7.10 container will be empty, so it will be the same as if it was dropped)

Do you forsee any issues there? Or is all edx-platform elasticsearch data ephemeral and/or able to automatically rebuild once a dev switches over to the 7.10 container?

Copy link
Copy Markdown
Contributor

@doctoryes doctoryes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 - this is the exact same diff I used for my work on this ticket: https://openedx.atlassian.net/browse/TNL-8485

@kdmccormick
Copy link
Copy Markdown
Contributor

We answered my question in standup -- the only thing this cache is used for is Course Search, which is rebuilt upon course publish, and is rarely used by developers. This shouldn't cause any dev disruption.

@connorhaugh connorhaugh merged commit c098d4c into master Aug 5, 2021
@connorhaugh connorhaugh deleted the chore--upgrade-devstack-edx-platform-es-to-7.10 branch August 5, 2021 19:19
nsprenkle pushed a commit that referenced this pull request Nov 21, 2023
chore: upgrade devstack edx-platform es to 7.10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants